Forward unrecognized slash commands to Codex#281
Conversation
Let Codex resolve slash commands that the adapter does not own.
benbrandt
left a comment
There was a problem hiding this comment.
@AlexandrSuhinin Maybe you all had this this way for a reason? But I think this change seems reasonable
We initially supported skills with the $ prefix: the same way they are supported in CLI. But anyway I'm not against this change, probably it is still better than throwing an error. |
|
Yeah I am also not sure why this is needed to resolve skills (in the PR description that is the motivation...) |
Pass slash commands not handled by the ACP adapter through as raw prompts, allowing Codex to resolve skills freely. Built-in commands continue to be handled locally.